Skip to content

Fix typos #4286

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Fix typos #4286

wants to merge 5 commits into from

Conversation

Marcofann
Copy link

This PR corrects several typos in comments across multiple Rust files in the codebase. These changes improve readability and maintain consistency without affecting functionality.

Files and Changes:

  • crates/relayer-cli/src/conclude.rs
    • Fixed "a error status" to "an error status."
  • tools/integration-test/src/tests/client_upgrade.rs
    • Corrected "an chain upgrade proposal" to "a chain upgrade proposal."
  • tools/test-framework/src/docs/walkthroughs/simple.rs
    • Fixed "an struct" to "a struct."
  • tools/test-framework/src/types/tagged/dual.rs
    • Corrected "an result tagged value" to "a result tagged value."
  • tools/test-framework/src/types/tagged/mono.rs
    • Corrected "an result tagged value" to "a result tagged value."

@Marcofann
typo conclude.rs
45e9112 
Signed-off-by: Marco <[email protected]>
@Marcofann
typo simple.rs
d891d1f 
Signed-off-by: Marco <[email protected]>
@Marcofann
typo dual.rs
db1e64f 
Signed-off-by: Marco <[email protected]>
@Marcofann
typo mono.rs
e305250 
Signed-off-by: Marco <[email protected]>
@Marcofann
typos client_upgrade.rs
94f00f0 
Signed-off-by: Marco <[email protected]>
@Marcofann
Copy link
Author

Hi everyone!

Just a friendly reminder about PR #4286: it corrects typos in comments across multiple Rust files:

  • crates/relayer-cli/src/conclude.rs: “a error status” → “an error status.”
  • tools/integration-test/src/tests/client_upgrade.rs: “an chain upgrade proposal” → “a chain upgrade proposal.”
  • tools/test-framework/src/docs/walkthroughs/simple.rs: “an struct” → “a struct.”
  • tools/test-framework/src/types/tagged/dual.rs: “an result tagged value” → “a result tagged value.”
  • tools/test-framework/src/types/tagged/mono.rs: “an result tagged value” → “a result tagged value.”

The branch is out-of-date with master and needs to be rebased/updated. At least one approving review is required by someone with write access. Could someone please update the branch and review when you have a moment? Thanks! 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@Marcofann